fix: repair StackChan voice assistant intercom external component - #93
Open
pfefferle wants to merge 4 commits into
Open
fix: repair StackChan voice assistant intercom external component#93pfefferle wants to merge 4 commits into
pfefferle wants to merge 4 commits into
Conversation
The n-IA-hane/esphome-intercom repo switched to date-based version tags and removed the old v3.1.0 tag, so the StackChan voice assistant config failed to compile with "couldn't find remote ref v3.1.0". Pin to v2026.5.0, which matches the ESPHome 2026.5.x line. Fixes m5stack#92
esp_aec declares AUTO_LOAD = ["audio_processor"], so the external component must be listed explicitly or ESPHome looks for a built-in audio_processor and fails with "Component not found: audio_processor".
pfefferle
changed the title
The i2s_audio_duplex component renamed the AEC attachment option from aec_id to processor_id (cv.use_id(AudioProcessor)). esp_aec is an AudioProcessor, so wire it via processor_id.
Contributor
|
I pinned intercom initially. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Repairs
common/stackchan-voice-assistant-base.yamlso it compiles against the currentn-IA-hane/esphome-intercomcomponent. The file was written for the old@v3.1.0API; three changes bring it up to date. Confirmed validating with ESPHome 2026.5.3.1. Broken git ref
The intercom repo switched to date-based tags and removed
v3.1.0:Pinned to
v2026.5.0(matches the ESPHome 2026.5.x line). Available upstream tags:v2026.5.0,v2026.6.0,v2026.6.1,v2026.6.2.2. Missing
audio_processorimportesp_aecdeclaresAUTO_LOAD = ["audio_processor"]; an auto-loaded external component must be listed explicitly or ESPHome looks for a built-in and fails:Added it to the import:
components: [i2s_audio_duplex, esp_aec, audio_processor].3.
aec_id→processor_idi2s_audio_duplexrenamed its AEC attachment option toprocessor_id(cv.use_id(AudioProcessor)):esp_aecis anAudioProcessor, so it's now wired viaprocessor_id: aec.Test plan
v2026.5.0exists viagit ls-remote --tags.esp_aec/__init__.pyhasAUTO_LOAD = ["audio_processor"]andi2s_audio_duplexacceptsprocessor_id(notaec_id).esp_aec, microphone (pre_aec), and speaker blocks validate unchanged againstv2026.5.0.Fixes #92